Add job suspend resume paths by scrappywyrm · Pull Request #29788 · Azure/azure-rest-api-specs

scrappywyrm · 2024-07-10T23:14:18Z

ARM (Control Plane) API Specification Update Pull Request

Tip

Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram

Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR

What's the purpose of this PR? Check the specific option that applies. This is mandatory!

New resource provider.
New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
Update existing version to fix OpenAPI spec quality issues in S360.
Other, please clarify:
- edit this with your clarification

Due diligence checklist

To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

Viewing API changes

For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures

If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

openapi-pipeline-app · 2024-07-10T23:14:21Z

Next Steps to Merge

Next steps that must be taken to merge this PR:

❌ This PR is in purview of the ARM review (label: ARMReview). This PR must get ARMSignedOff label from an ARM reviewer.
This PR has ARMChangesRequested label. Please address or respond to feedback from the ARM API reviewer.
When you are ready to continue the ARM API review, please remove the ARMChangesRequested label.
Automation should then add WaitForARMFeedback label.
❗If you don't have permissions to remove the label, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories.
For details of the ARM review, see aka.ms/azsdk/pr-arm-review
❌ The required check named Automated merging requirements met has failed. This is the final check that must pass. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide. In addition, refer to step 4 in the PR workflow diagram

openapi-pipeline-app · 2024-07-10T23:14:22Z

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]

There are no breaking changes.

️❌Breaking Change(Cross-Version): 8 Errors, 6 Warnings failed [Detail]

Compared specs (v0.10.12)	new version	base version
Jobs.json	2024-08-02-preview(9c0fa31)	2024-03-01(main)
Jobs.json	2024-08-02-preview(9c0fa31)	2024-02-02-preview(main)

The following breaking changes are detected by comparison with the latest stable version:

Rule	Message
`❌` 1023 - TypeFormatChanged	The new version has a different format 'arm-id' than the previous one ''. New: common-types/resource-management/v5/types.json#L14:9 Old: common-types/resource-management/v3/types.json#L14:9
`❌` 1023 - TypeFormatChanged	The new version has a different format 'arm-id' than the previous one ''. New: common-types/resource-management/v5/types.json#L14:9 Old: common-types/resource-management/v3/types.json#L14:9
`❌` 1023 - TypeFormatChanged	The new version has a different format 'arm-id' than the previous one ''. New: common-types/resource-management/v5/types.json#L14:9 Old: common-types/resource-management/v3/types.json#L14:9
`❌` 1023 - TypeFormatChanged	The new version has a different format 'arm-id' than the previous one ''. New: common-types/resource-management/v5/types.json#L14:9 Old: common-types/resource-management/v3/types.json#L14:9
`❌` 1023 - TypeFormatChanged	The new version has a different format 'arm-id' than the previous one ''. New: common-types/resource-management/v5/types.json#L14:9 Old: common-types/resource-management/v3/types.json#L14:9
`❌` 1023 - TypeFormatChanged	The new version has a different format 'uuid' than the previous one ''. New: common-types/resource-management/v5/types.json#L204:9 Old: common-types/resource-management/v3/types.json#L204:9
`❌` 1023 - TypeFormatChanged	The new version has a different format 'uuid' than the previous one ''. New: common-types/resource-management/v5/types.json#L210:9 Old: common-types/resource-management/v3/types.json#L209:9
`❌` 1023 - TypeFormatChanged	The new version has a different format 'arm-id' than the previous one ''. New: common-types/resource-management/v5/types.json#L14:9 Old: common-types/resource-management/v3/types.json#L14:9
⚠️ 1038 - AddedPath	The new version is adding a path that was not found in the old version. New: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L629:5
⚠️ 1038 - AddedPath	The new version is adding a path that was not found in the old version. New: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L685:5
⚠️ 1040 - AddedReadOnlyPropertyInResponse	The new version has a new read-only property 'runningState' in response that was not found in the old version. New: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L892:11 Old: Microsoft.App/stable/2024-03-01/Jobs.json#L764:11

The following breaking changes are detected by comparison with the latest preview version:

Rule	Message
⚠️ 1038 - AddedPath	The new version is adding a path that was not found in the old version. New: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L629:5
⚠️ 1038 - AddedPath	The new version is adding a path that was not found in the old version. New: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L685:5
⚠️ 1040 - AddedReadOnlyPropertyInResponse	The new version has a new read-only property 'runningState' in response that was not found in the old version. New: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L892:11 Old: Microsoft.App/preview/2024-02-02-preview/Jobs.json#L780:11

️❌LintDiff: 4 Errors, 6 Warnings failed [Detail]

Compared specs (v2.2.2)	new version	base version
package-preview-2024-08	package-preview-2024-08(9c0fa31)	package-preview-2024-08(release-app-Microsoft.App-2024-08-02-preview)

[must fix]The following errors/warnings are introduced by current PR:

Rule	Message	Related RPC [For API reviewers]
`❌` PostResponseCodes	200 return code does not have a schema specified. LRO POST must have a 200 return code if only if the final response is intended to have a schema, if not the 200 return code must not be specified. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L630	RPC-Async-V1-11, RPC-Async-V1-14
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L670
`❌` PostResponseCodes	200 return code does not have a schema specified. LRO POST must have a 200 return code if only if the final response is intended to have a schema, if not the 200 return code must not be specified. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L686	RPC-Async-V1-11, RPC-Async-V1-14
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L726
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L638
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L641
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L644
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L694
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L697
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L700

The following errors/warnings exist before current PR submission:

Only 30 items are listed, please refer to log for more details.

Rule	Message
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L186
`❌` DeleteResponseCodes	Long-running delete operations must have responses with 202, 204 and default return codes. They also must have no other response codes. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L206
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L245
`❌` PatchBodyParametersSchema	Properties of a PATCH request body must not be required, property:type. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L281
`❌` PatchBodyParametersSchema	Properties of a PATCH request body must not have default value, property:triggerType. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L281
`❌` PatchBodyParametersSchema	Properties of a PATCH request body must not be required, property:triggerType. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L281
`❌` PatchBodyParametersSchema	Properties of a PATCH request body must not be required, property:replicaTimeout. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L281
`❌` PatchBodyParametersSchema	Properties of a PATCH request body must not be required, property:cronExpression. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L281
`❌` PatchBodyParametersSchema	Properties of a PATCH request body must not have default value, property:maxExecutions. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L281
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L307
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L367
`❌` PostResponseCodes	200 return code does not have a schema specified. LRO POST must have a 200 return code if only if the final response is intended to have a schema, if not the 200 return code must not be specified. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L383
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L421
`❌` LroErrorContent	Error response content of long running operations must follow the error schema provided in the common types v2 and above. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L475
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L27
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L30
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L66
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L69
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L72
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L108
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L111
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L117
⚠️ SummaryAndDescriptionMustNotBeSame	The summary and description values should not be same. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L140
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L149
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L152
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L167
⚠️ SummaryAndDescriptionMustNotBeSame	The summary and description values should not be same. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L206
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L215
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L218
⚠️ LatestVersionOfCommonTypesMustBeUsed	Use the latest version v5 of types.json. Location: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L224

️️✔️Avocado succeeded [Detail] [Expand]

Validation passes for Avocado.

️️✔️SwaggerAPIView succeeded [Detail] [Expand]

️️✔️TypeSpecAPIView succeeded [Detail] [Expand]

️❌ModelValidation: 4 Errors, 0 Warnings failed [Detail]

Rule	Message
`❌` RESPONSE_SCHEMA_NOT_IN_SPEC	Response statusCode 200 for operation Jobs_Resume has response body provided in the example, however the response does not have a "schema" defined in the swagger spec. Url: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L655:22 ExampleUrl: preview/2024-08-02-preview/examples/Jobs_Resume.json#L8:16
`❌` RESPONSE_SCHEMA_NOT_IN_SPEC	Response statusCode 200 for operation Jobs_Suspend has response body provided in the example, however the response does not have a "schema" defined in the swagger spec. Url: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L711:22 ExampleUrl: preview/2024-08-02-preview/examples/Jobs_Suspend.json#L8:16
`❌` RESPONSE_SCHEMA_NOT_IN_SPEC	Response statusCode 200 for operation Jobs_Resume has response body provided in the example, however the response does not have a "schema" defined in the swagger spec. Url: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L655:22 ExampleUrl: preview/2024-08-02-preview/examples/Jobs_Resume.json#L8:16
`❌` RESPONSE_SCHEMA_NOT_IN_SPEC	Response statusCode 200 for operation Jobs_Suspend has response body provided in the example, however the response does not have a "schema" defined in the swagger spec. Url: Microsoft.App/preview/2024-08-02-preview/Jobs.json#L711:22 ExampleUrl: preview/2024-08-02-preview/examples/Jobs_Suspend.json#L8:16

️️✔️SemanticValidation succeeded [Detail] [Expand]

Validation passes for SemanticValidation.

️️✔️SpellCheck succeeded [Detail] [Expand]

Validation passes for SpellCheck.

️️✔️PR Summary succeeded [Detail] [Expand]

Validation passes for Summary.

️️✔️Automated merging requirements met succeeded [Detail] [Expand]

_{Posted by Swagger Pipeline | How to fix these errors?}

openapi-pipeline-app · 2024-07-10T23:14:22Z

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]

 Please click here to preview with your @microsoft account.

️⚠️ azure-sdk-for-python warning [Detail]

FAQ

⚠️Warning in generating from 8f5ec5aca5ad161406a1396767e6c768aa00048d. SDK Automation 14.0.0

command	sh scripts/automation_init.sh ../azure-sdk-for-python_tmp/initInput.json ../azure-sdk-for-python_tmp/initOutput.json
cmderr	[automation_init.sh] W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target CNF (main/cnf/Commands-amd64) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target CNF (main/cnf/Commands-all) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target CNF (main/cnf/Commands-amd64) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] W: Target CNF (main/cnf/Commands-all) is configured multiple times in /etc/apt/sources.list.d/azure-cli.list:1 and /etc/apt/sources.list.d/azure-cli.sources:1
cmderr	[automation_init.sh] WARNING: Skipping azure-nspkg as it is not installed.
cmderr	[automation_init.sh] npm notice
cmderr	[automation_init.sh] npm notice New minor version of npm available! 10.7.0 -> 10.8.2
cmderr	[automation_init.sh] npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.8.2
cmderr	[automation_init.sh] npm notice To update run: npm install -g npm@10.8.2
cmderr	[automation_init.sh] npm notice
command	sh scripts/automation_generate.sh ../azure-sdk-for-python_tmp/generateInput.json ../azure-sdk-for-python_tmp/generateOutput.json

️✔️azure-mgmt-appcontainers [Preview SDK Changes]

️️✔️ azure-sdk-for-java succeeded [Detail] [Expand]

FAQ

️✔️Succeeded in generating from 8f5ec5aca5ad161406a1396767e6c768aa00048d. SDK Automation 14.0.0

command	./eng/automation/init.sh ../azure-sdk-for-java_tmp/initInput.json ../azure-sdk-for-java_tmp/initOutput.json
command	./eng/automation/generate.py ../azure-sdk-for-java_tmp/generateInput.json ../azure-sdk-for-java_tmp/generateOutput.json

️✔️azure-resourcemanager-appcontainers [Preview SDK Changes]
```
info	[Changelog]
```

️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]

FAQ

️✔️Succeeded in generating from 8f5ec5aca5ad161406a1396767e6c768aa00048d. SDK Automation 14.0.0

command	sh ./eng/scripts/automation_init.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
command	generator automation-v2 ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json

️✔️sdk/resourcemanager/appcontainers/armappcontainers [Preview SDK Changes]

️️✔️ azure-sdk-for-js succeeded [Detail] [Expand]

FAQ

️✔️Succeeded in generating from 8f5ec5aca5ad161406a1396767e6c768aa00048d. SDK Automation 14.0.0

command	sh .scripts/automation_init.sh ../azure-sdk-for-js_tmp/initInput.json ../azure-sdk-for-js_tmp/initOutput.json
warn	Warning: File azure-sdk-for-js_tmp/initOutput.json not found to read. Please re-run the pipeline if the error is transitient error or report this issue through https://aka.ms/azsdk/support/specreview-channel.
command	sh .scripts/automation_generate.sh ../azure-sdk-for-js_tmp/generateInput.json ../azure-sdk-for-js_tmp/generateOutput.json

️✔️@azure/arm-appcontainers [Preview SDK Changes]

️❌ azure-resource-manager-schemas failed [Detail]

FAQ

❌Code Generator Failed in generating from 8f5ec5aca5ad161406a1396767e6c768aa00048d. Schema Automation 14.0.0

command	.sdkauto/initScript.sh ../azure-resource-manager-schemas_tmp/initInput.json ../azure-resource-manager-schemas_tmp/initOutput.json
cmderr	[initScript.sh] Submodule 'bicep-types-az' (https://github.com/Azure/bicep-types-az) registered for path 'bicep-types-az'
cmderr	[initScript.sh] Cloning into '/mnt/vss/_work/1/s/azure-resource-manager-schemas/bicep-types-az'...
cmderr	[initScript.sh] Submodule 'bicep-types' (https://github.com/Azure/bicep-types) registered for path 'bicep-types-az/bicep-types'
cmderr	[initScript.sh] Cloning into '/mnt/vss/_work/1/s/azure-resource-manager-schemas/bicep-types-az/bicep-types'...
cmderr	[initScript.sh] npm notice
cmderr	[initScript.sh] npm notice New minor version of npm available! 10.7.0 -> 10.8.2
cmderr	[initScript.sh] npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.8.2
cmderr	[initScript.sh] npm notice To update run: npm install -g npm@10.8.2
cmderr	[initScript.sh] npm notice
error	Script return with result [failed] code [1] signal [null] cwd [azure-resource-manager-schemas]: .sdkauto/initScript.sh
warn	Warning: File azure-resource-manager-schemas_tmp/initOutput.json not found to read. Please re-run the pipeline if the error is transitient error or report this issue through https://aka.ms/azsdk/support/specreview-channel.
command	.sdkauto/generateScript.sh ../azure-resource-manager-schemas_tmp/generateInput.json ../azure-resource-manager-schemas_tmp/generateOutput.json
cmderr	[generateScript.sh] /mnt/vss/_work/1/a/unified-pipeline-runtime/common/temp/node_modules/.pnpm/ts-node@9.0.0_typescript@5.3.3/node_modules/ts-node/src/index.ts:500
cmderr	[generateScript.sh]     return new TSError(diagnosticText, diagnosticCodes)
cmderr	[generateScript.sh]            ^
cmderr	[generateScript.sh] TSError: ⨯ Unable to compile TypeScript:
cmderr	[generateScript.sh] cmd/generateall.ts(6,20): error TS2307: Cannot find module 'colors' or its corresponding type declarations.
cmderr	[generateScript.sh] cmd/generateall.ts(10,19): error TS2307: Cannot find module 'yargs' or its corresponding type declarations.
cmderr	[generateScript.sh] cmd/generateall.ts(11,18): error TS2307: Cannot find module 'path' or its corresponding type declarations.
cmderr	[generateScript.sh] cmd/generateall.ts(13,35): error TS2307: Cannot find module 'fs' or its corresponding type declarations.
cmderr	[generateScript.sh] cmd/generateall.ts(14,23): error TS2307: Cannot find module 'strip-ansi' or its corresponding type declarations.
cmderr	[generateScript.sh] cmd/generateall.ts(42,9): error TS2584: Cannot find name 'console'. Do you need to change your target library? Try changing the 'lib' compiler option to include 'dom'.
cmderr	[generateScript.sh] cmd/generateall.ts(53,9): error TS2584: Cannot find name 'console'. Do you need to change your target library? Try changing the 'lib' compiler option to include 'dom'.
cmderr	[generateScript.sh] cmd/generateall.ts(69,67): error TS7006: Parameter 'x' implicitly has an 'any' type.
cmderr	[generateScript.sh] cmd/generateall.ts(70,49): error TS7006: Parameter 'f' implicitly has an 'any' type.
cmderr	[generateScript.sh] cmd/generateall.ts(94,21): error TS2584: Cannot find name 'console'. Do you need to change your target library? Try changing the 'lib' compiler option to include 'dom'.
cmderr	[generateScript.sh] cmd/generateall.ts(105,21): error TS2584: Cannot find name 'console'. Do you need to change your target library? Try changing the 'lib' compiler option to include 'dom'.
cmderr	[generateScript.sh] cmd/generateall.ts(106,21): error TS2584: Cannot find name 'console'. Do you need to change your target library? Try changing the 'lib' compiler option to include 'dom'.
cmderr	[generateScript.sh] cmd/generateall.ts(157,13): error TS2591: Cannot find name 'process'. Do you need to install type definitions for node? Try `npm i --save-dev @types/node` and then add 'node' to the types field in your tsconfig.
cmderr	[generateScript.sh]     at createTSError (/mnt/vss/_work/1/a/unified-pipeline-runtime/common/temp/node_modules/.pnpm/ts-node@9.0.0_typescript@5.3.3/node_modules/ts-node/src/index.ts:500:12)
cmderr	[generateScript.sh]     at reportTSError (/mnt/vss/_work/1/a/unified-pipeline-runtime/common/temp/node_modules/.pnpm/ts-node@9.0.0_typescript@5.3.3/node_modules/ts-node/src/index.ts:504:19)
cmderr	[generateScript.sh]     at getOutput (/mnt/vss/_work/1/a/unified-pipeline-runtime/common/temp/node_modules/.pnpm/ts-node@9.0.0_typescript@5.3.3/node_modules/ts-node/src/index.ts:739:36)
cmderr	[generateScript.sh]     at Object.compile (/mnt/vss/_work/1/a/unified-pipeline-runtime/common/temp/node_modules/.pnpm/ts-node@9.0.0_typescript@5.3.3/node_modules/ts-node/src/index.ts:955:32)
cmderr	[generateScript.sh]     at Module.m._compile (/mnt/vss/_work/1/a/unified-pipeline-runtime/common/temp/node_modules/.pnpm/ts-node@9.0.0_typescript@5.3.3/node_modules/ts-node/src/index.ts:1043:43)
cmderr	[generateScript.sh]     at Module._extensions..js (node:internal/modules/cjs/loader:1422:10)
cmderr	[generateScript.sh]     at Object.require.extensions.<computed> [as .ts] (/mnt/vss/_work/1/a/unified-pipeline-runtime/common/temp/node_modules/.pnpm/ts-node@9.0.0_typescript@5.3.3/node_modules/ts-node/src/index.ts:1046:12)
cmderr	[generateScript.sh]     at Module.load (node:internal/modules/cjs/loader:1203:32)
cmderr	[generateScript.sh]     at Function.Module._load (node:internal/modules/cjs/loader:1019:12)
cmderr	[generateScript.sh]     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:128:12)
error	Script return with result [failed] code [1] signal [null] cwd [azure-resource-manager-schemas]: .sdkauto/generateScript.sh
warn	Warning: Package processing is skipped as the SDK generation fails. Please look into the above generation errors or report this issue through https://aka.ms/azsdk/support/specreview-channel.

️️✔️ azure-powershell succeeded [Detail] [Expand]

FAQ

️✔️Succeeded in generating from 8f5ec5aca5ad161406a1396767e6c768aa00048d. SDK Automation 14.0.0

command	sh ./tools/SwaggerCI/init.sh ../azure-powershell_tmp/initInput.json ../azure-powershell_tmp/initOutput.json
command	pwsh ./tools/SwaggerCI/psci.ps1 ../azure-powershell_tmp/generateInput.json ../azure-powershell_tmp/generateOutput.json

️✔️Az.app.DefaultTag [Preview SDK Changes]

_{Posted by Swagger Pipeline | How to fix these errors?}

openapi-pipeline-app · 2024-07-10T23:14:23Z

Generated ApiView

Language	Package Name	ApiView Link
Go	sdk/resourcemanager/appcontainers/armappcontainers	https://apiview.dev/Assemblies/Review/77a1e1c7ff4c495e8e1c9cf369fdf74a?revisionId=65c2736db80749d3af482635a30b7e33
Java	azure-resourcemanager-appcontainers	https://apiview.dev/Assemblies/Review/7da9ac1e2db64477ada2ba8bbb71169b?revisionId=e04161114d5a458baac9a7c52782b91f
JavaScript	@azure/arm-appcontainers	https://apiview.dev/Assemblies/Review/ba41cd9978b542efb0688a072b2319a1?revisionId=f743eb270335457ba45c0d7327c89282
Swagger	Microsoft.App	https://apiview.dev/Assemblies/Review/53999dbad23b45cba7812a00a713cab8?revisionId=3822faaf5f8946f0bedcec1bd5087820

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json

fix runningState and add azure-asyncoperation

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json

raosuhas · 2024-07-16T23:40:55Z

You have similar flagged errors in your LintDiff validations here : https://github.com/Azure/azure-rest-api-specs/pull/29788/checks?check_run_id=27473684037

openapi-pipeline-app · 2024-07-18T19:26:04Z

PR validation pipeline can not start as the pull request is not merged or mergeable - most likely it has merge conflicts.

…job-suspend-resume

scrappywyrm · 2024-07-23T22:10:00Z

cc: @raosuhas hi, could you please take a look at the recent changes? thank you!!

...tion/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/examples/Jobs_Resume.json

smzeng · 2024-07-31T19:35:04Z

i think overall looks good, maybe also update the history.rst file and also add tests

scrappywyrm · 2024-07-31T21:22:50Z

i think overall looks good, maybe also update the history.rst file and also add tests

thanks @smzeng ....history.rst and tests are for cli PRs i believe?

smzeng · 2024-07-31T22:33:10Z

i think overall looks good, maybe also update the history.rst file and also add tests

thanks @smzeng ....history.rst and tests are for cli PRs i believe?

youre totally right, thats my bad haha. No wonder i couldn't find the file when i tried to link it earlier. Looks good then!

…job-suspend-resume

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json

jijohn14 · 2024-08-15T22:47:57Z

Change will be added to next preview api-version

bgashirabake@gmail.com added 2 commits July 10, 2024 16:00

add fields

4b27bd8

add

9c0fa31

scrappywyrm requested review from Juliehzl and jijohn14 as code owners July 10, 2024 23:14

microsoft-github-policy-service bot added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Jul 10, 2024

AzureRestAPISpecReview added ARMReview resource-manager WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Jul 10, 2024

jijohn14 reviewed Jul 15, 2024

View reviewed changes

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json Show resolved Hide resolved

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json Show resolved Hide resolved

scrappywyrm and others added 4 commits July 15, 2024 12:17

Update Jobs.json

31e6b1f

fix runningState and add azure-asyncoperation

Update Jobs_Resume.json

4d52e9c

Update Jobs_Suspend.json

d7c5bf7

add azure-asyncoperation

588b532

This was referenced Jul 15, 2024

[AutoPR azure-resourcemanager-appcontainers] Add job suspend resume paths azure-sdk/azure-sdk-for-java#11215

Closed

[AutoPR azure-mgmt-appcontainers] Add job suspend resume paths azure-sdk/azure-sdk-for-python#7638

Closed

raosuhas reviewed Jul 16, 2024

View reviewed changes

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json Outdated Show resolved Hide resolved

raosuhas reviewed Jul 16, 2024

View reviewed changes

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json Outdated Show resolved Hide resolved

raosuhas added the ARMChangesRequested label Jul 16, 2024

openapi-pipeline-app bot removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Jul 16, 2024

bgashirabake@gmail.com added 2 commits July 17, 2024 15:38

fix lintdiff

77801a1

fix

7ab0493

scrappywyrm force-pushed the add-job-suspend-resume branch from cb2178b to 4117847 Compare July 18, 2024 19:25

scrappywyrm requested review from vicancy and removed request for a team July 18, 2024 19:25

openapi-pipeline-app bot added the BreakingChange-Go-Sdk-Suppression label Jul 18, 2024

scrappywyrm force-pushed the add-job-suspend-resume branch from 4117847 to cb2178b Compare July 18, 2024 19:40

openapi-pipeline-app bot removed the BreakingChange-Go-Sdk-Suppression label Jul 18, 2024

fix

c7ab3d8

scrappywyrm force-pushed the add-job-suspend-resume branch from 70d583a to c7ab3d8 Compare July 19, 2024 00:49

scrappywyrm requested a review from raosuhas July 22, 2024 18:38

fix

2ac7e6c

scrappywyrm force-pushed the add-job-suspend-resume branch from 19f33c0 to 2ac7e6c Compare July 23, 2024 20:19

bgashirabake@gmail.com and others added 2 commits July 23, 2024 13:52

add

3e29350

Merge branch 'release-app-Microsoft.App-2024-08-02-preview' into add-…

678f0ea

…job-suspend-resume

scrappywyrm requested a review from jijohn14 July 23, 2024 22:10

jijohn14 suggested changes Jul 30, 2024

View reviewed changes

...tion/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/examples/Jobs_Resume.json Outdated Show resolved Hide resolved

Update Jobs_Resume.json

1b17a0f

Merge branch 'release-app-Microsoft.App-2024-08-02-preview' into add-…

6a00d7a

…job-suspend-resume

jijohn14 approved these changes Aug 5, 2024

View reviewed changes

p-bouchon reviewed Aug 6, 2024

View reviewed changes

specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json Outdated Show resolved Hide resolved

scrappywyrm requested a review from p-bouchon August 6, 2024 16:28

fix JobName parameter

d127169

azure-sdk added the BreakingChange-Python-Sdk label Aug 6, 2024

p-bouchon approved these changes Aug 6, 2024

View reviewed changes

jijohn14 closed this Aug 15, 2024

scrappywyrm mentioned this pull request Aug 22, 2024

Add job suspend resume #30304

Merged

Conversation

scrappywyrm commented Jul 10, 2024

ARM (Control Plane) API Specification Update Pull Request

PR review workflow diagram

Purpose of this PR

Due diligence checklist

Additional information

Getting help

Uh oh!

openapi-pipeline-app bot commented Jul 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Next Steps to Merge

Uh oh!

openapi-pipeline-app bot commented Jul 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Swagger Validation Report

Uh oh!

openapi-pipeline-app bot commented Jul 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Swagger Generation Artifacts

Uh oh!

openapi-pipeline-app bot commented Jul 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Generated ApiView

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

raosuhas commented Jul 16, 2024

Uh oh!

openapi-pipeline-app bot commented Jul 18, 2024

Uh oh!

scrappywyrm commented Jul 23, 2024

Uh oh!

Uh oh!

smzeng commented Jul 31, 2024

Uh oh!

scrappywyrm commented Jul 31, 2024

Uh oh!

smzeng commented Jul 31, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

jijohn14 commented Aug 15, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

openapi-pipeline-app bot commented Jul 10, 2024 •

edited

Loading

openapi-pipeline-app bot commented Jul 10, 2024 •

edited

Loading

openapi-pipeline-app bot commented Jul 10, 2024 •

edited

Loading

openapi-pipeline-app bot commented Jul 10, 2024 •

edited

Loading

smzeng commented Jul 31, 2024 •

edited

Loading